Skip to content

Codecov integration #544

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Codecov integration #544

wants to merge 10 commits into from

Conversation

buenaflor
Copy link
Contributor

#skip-changelog

@codecov
Copy link

codecov bot commented Aug 9, 2023 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (main@429f2c7). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #544   +/-   ##
=======================================
  Coverage        ?   56.79%           
  Complexity      ?      476           
=======================================
  Files           ?      105           
  Lines           ?     3465           
  Branches        ?      373           
=======================================
  Hits            ?     1968           
  Misses          ?     1395           
  Partials        ?      102

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@buenaflor
Copy link
Contributor Author

buenaflor commented Aug 10, 2023 •
edited
Loading

@romtsn @markushi
The coverage report: https://app.codecov.io/github/getsentry/sentry-android-gradle-plugin/commit/20a12417f06a7063be29248245773523145e3828/tree

Does this look right? Also lmk if I missed any modules or should exclude anything.
note: (current state of the code is just for testing)

@getsantry
Copy link

getsantry bot commented Oct 18, 2023

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot added the Stale label Oct 18, 2023
@romtsn romtsn removed the Stale label Oct 19, 2023
@buenaflor
Copy link
Contributor Author

buenaflor commented Dec 13, 2023 •
edited
Loading

fyi, I will continue this PR soon

EDIT: I did not

@buenaflor
Copy link
Contributor Author

Just commenting that I haven't forgotten this PR, maybe makes sense to restart it since I forgot almost all of the context of it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markushi markushi Awaiting requested review from markushi markushi will be requested when the pull request is marked ready for review markushi is a code owner

@romtsn romtsn Awaiting requested review from romtsn romtsn will be requested when the pull request is marked ready for review romtsn is a code owner

@adinauer adinauer Awaiting requested review from adinauer adinauer will be requested when the pull request is marked ready for review adinauer is a code owner

@stefanosiano stefanosiano Awaiting requested review from stefanosiano stefanosiano will be requested when the pull request is marked ready for review stefanosiano is a code owner

@lcian lcian Awaiting requested review from lcian lcian will be requested when the pull request is marked ready for review lcian is a code owner

@runningcode runningcode Awaiting requested review from runningcode runningcode will be requested when the pull request is marked ready for review runningcode is a code owner

@chromy chromy Awaiting requested review from chromy chromy will be requested when the pull request is marked ready for review chromy is a code owner

@rbro112 rbro112 Awaiting requested review from rbro112 rbro112 will be requested when the pull request is marked ready for review rbro112 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@buenaflor @romtsn